Specific event page #22
Conversation
Deploying with
|
| Latest commit: |
57bbdc9
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://a76ea64d.tickets.pages.dev |
| Branch Preview URL: | https://feature-event-view.tickets.pages.dev |
There was a problem hiding this comment.
Le pegue una mirada, a nivel de código solo estos temas ATM.
(Acabo de cachar q estas mockeando algunas cosas... obvia lode graphql...pero en vola fase 2 :P )
Cuando deploye y testie en mi local, cache q en pantallas grandes, los elementos siguen creciendo. El wrapper de la página debería tener un max-width (creo q lo deje en el figma IIRC, 1 seg)
(Al ojo, 1360px...deberíamos darle lo q sea q tailwind tenga por ahi cerca, y padding laterales al contendor 🙏 )
|
|
||
| export type OrganizersTypes = { | ||
| title: string; | ||
| organizers: Organizer[]; |
There was a problem hiding this comment.
En ves de Organizer ( en el componente Attendees, que estamos usando type Attendees) deberíamos usar los tipos de GraphQl, por si cambian tener menos problemas.
Podríamos referenciar un UserFragment por ejemplo ...
Igual este mensaje es un poco nitpicky ....sientete libre de ignorar, pero IMO interesante pa pensarlo :)
| }, | ||
| ]; | ||
|
|
||
| export const event = { |
There was a problem hiding this comment.
en el futuro, deberiamos usar faker para los fixtures
En este caso. es fixture un nombre protegido de esta carpeta? onda podemos crear este archivo aca (dentro de /app digo?)
There was a problem hiding this comment.
No entendí muy bien la pregunta 😢
| name: string; | ||
| } | ||
|
|
||
| type EventClass = { |
There was a problem hiding this comment.
Para estos tipos deberiamos usar los de src/api/gql/graphql.ts
There was a problem hiding this comment.
(Nos ahorramos que diverjan en lo que cambie la API)
No description provided.